New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix translation catalog #10933
Fix translation catalog #10933
Conversation
What about now @LouiseBonnard ? |
app/Resources/translations/default/ModulesEmailsubscriptionAdmin.xlf
Outdated
Show resolved
Hide resolved
app/Resources/translations/default/ModulesEmailsubscriptionAdmin.xlf
Outdated
Show resolved
Hide resolved
app/Resources/translations/default/ModulesFaviconnotificationboAdmin.xlf
Outdated
Show resolved
Hide resolved
@@ -76,7 +76,7 @@ | |||
<thead> | |||
<tr> | |||
<th class="fixed-width-xs"><span class="title_box">{{ 'ID'|trans({}, 'Admin.Global') }}</span></th> | |||
<th><span class="title_box">{{ 'IP address'|trans({}, 'messages') }}</span></th> | |||
<th><span class="title_box">{{ 'IP address'|trans }}</span></th> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IP Address instead. :-(
app/Resources/translations/default/ModulesFaviconnotificationboAdmin.xlf
Outdated
Show resolved
Hide resolved
@@ -435,6 +561,12 @@ File: modules/autoupgrade/classes/UpgradeTools/CoreUpgrader/CoreUpgrader.php:479 | |||
</file> | |||
<file original="modules/autoupgrade/classes/UpgradeTools/CoreUpgrader/CoreUpgrader16.php" source-language="en-US" target-language="en" datatype="plaintext"> | |||
<body> | |||
<trans-unit id="618bfc1f9ba85d07157a388208fe3ff8"> | |||
<source>Settings file updated</source> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not a better wording from an existing translation string, like Settings updated in Admin.Global? Formulation is clearer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that we are specifically talking about a settings file, and not settings in general.
Wdyt @Quetzacoalt91 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly. We are talking about the file config/settings.inc.php available on PS 1.6
File: modules/autoupgrade/classes/UpgradePage.php:333</note> | ||
</trans-unit> | ||
<trans-unit id="c12803674a35ccc28e5da00a50a84591"> | ||
<source><![CDATA[Product images do not appear in the front-office? Try regenerating the thumbnails in Preferences > Images]]></source> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What I get on Crowdin is that: Product images do not appear in the front-office? Try regenerating the thumbnails in Preferences > Images, should we change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The wording looks the same to me. Am I missing something? The <![CDATA[]]>
around is syntactic sugar to prevent the literal >
inside the wording from breaking the XML file. You can disregard that, it won't appear in Crowdin.
File: modules/autoupgrade/classes/UpgradePage.php:340</note> | ||
</trans-unit> | ||
<trans-unit id="a8192fd08a431bf547ebf9a3eaa9b6bb"> | ||
<source>Javascript error (parseJSON) detected for action </source> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Watch the space at the end. ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Quetzacoalt91 was it left there on purpose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the action name is concatenated. Note I did not change this string with the latest versions of the module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok we'll leave it as is then 👍
File: modules/autoupgrade/views/templates/block/channelInfo.twig:5</note> | ||
</trans-unit> | ||
<trans-unit id="4e140ba723a03baa6948340bf90e2ef6"> | ||
<source>Name:</source> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already in Admin.Global.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@LouiseBonnard do you approve? |
ping @LouiseBonnard |
Approved by @LouiseBonnard |
This change is